[Quartz Bug 663182] NSImage throws an exception from _gtk_quartz_create_image_from_pixbuf()"

If a zero-sized NSImage calls lockfocus it throws an exception. Make sure that the image isn't zero-sized before calling lockfocus, and log a warning and return NULL if it is zero-sized.

Have the quartz version of gtk_drag_begin_idle return FALSE if the returned image is NULL.

Revert "[Quartz Bug 663182] NSImage throws an exception from _gtk_quartz_create_image_from_pixbuf()". Utterly screwed up. Sorry.

This reverts commit 8216324e4b13f0349ed3c0f854d41e6b32d1f0cb.
and          commit 3243e6955cd4014345170fdd9246a13eacad86dd.

Fix accidental paste error in 8216324

[Quartz Bug 663182] NSImage throws an exception from _gtk_quartz_create_image_from_pixbuf()

If _gtk_quartz_create_image_from_pixbuf is given a pixbuf with size 0, 0
or which produces an NSImage with size 0.0, 0.0, it throws an exception
which Gtk doesn't handle.

your changes. Lines starting

gdk: exclude MOD1 from the virtual modifier mapping

which effectively nails down the MOD1 == ALT assumption that is valid
in all other parts of GTK+. After the modifier abstraction fixes for
OSX, the virtual mapping is now (correctly) used in more places, and
caused problems with the common default PC keyboard layout on X11 that
colocates ALT and META on the same key.

Print preview: make cancel work

This makes cancels of a print preview work in the same way it
works for non-previews, i.e., the operation stops at the next
page boundary.

https://bugzilla.gnome.org/show_bug.cgi?id=662160

quartz: handle recursive CFRunLoops

Fixes e.g. crashs when dropping from finder.

Turn the "getting_events" boolean into a counter to handle poll_func()
being called recursively, and track the loop depth correctly by
changing its counter before bailing out in run_loop_observer_callback().
This way we reallocate our autorelease pool at the right time, and
don't kill memory that is still in use by outer run loops.

Also drain, not release the pool, just for some defensive forward
compatibility.
(cherry picked from commit ef9a92d225f7257f56c3a8b7b28ddd9a734e38e6)

gtk: add annotations for some GtkRadio* constructors

Since constructors like gtk_radio_button_new_from_widget take an
instance of their type as the first argument, the gi scanner interprets
them as methods.

https://bugzilla.gnome.org/show_bug.cgi?id=661858

quartz: use get_toplevel_from_ns_event in one more place

Fixup needed due to refactoring in GTK+-3.0.

quartz: make gdk_quartz_osx_version conform to coding style

quartz: make test_resize () conform to coding style

quartz: Process motion events within windows bounds without window set

When an NSEvent does not have the window field set, we already assumed
the event was not for us and discarded it.  But for NSMouseMoved events
we now make an exception, because such events generated after
using/clicking the main menu bar have the window field set to NULL while
the application window still has focus.

We used to experience a loss of motion events after using the menu bar,
this could be seen in buttons that stopped prelighting and first
clicks often being ignored unless you clicked somewhere else first.
These issues are fixed by this patch.

quartz: Factor out toplevel from NSEvent code into function

quartz: Separate out screen_point conversion in function

Reduces code duplication and confusion.

quartz: remove unused variable

menuitem: Always treat submenus as popup

Submenus are not actually attached to a menubar, but styling them
as if they were imposes some problems (at least with Adwaita).
Just using GDK_WINDOW_TYPE_HINT_POPUP_MENU for submenus instead
fixes the issue.

https://bugzilla.gnome.org/show_bug.cgi?id=662691

Allow fallback for input method modules

Accept a :-separated list of module names in GTK_IM_MODULE and
the corresponding setting, to deal a bit better with broken
situations.

https://bugzilla.gnome.org/show_bug.cgi?id=603559

gtk: add annotations for some GtkSymbolicColor constructors

https://bugzilla.gnome.org/show_bug.cgi?id=661859

x11: Fix window manager change detection

Since the wmspec_check_window doesn't have a corresponding GdkWindow we can't
rely on the get_event_window() return value to get the XID from. Just use the
XID from the XEvent directly.

https://bugzilla.gnome.org/show_bug.cgi?id=662953

Mark gtk_dialog_new_with_buttons as NULL-terminated

https://bugzilla.gnome.org/show_bug.cgi?id=663073

Pop up context menus at the right device

When multiple pointers are in play, we need to be careful
not to loose track of the device between receiving a button
press and popping up a menu.

https://bugzilla.gnome.org/show_bug.cgi?id=663396

GtkWindow: Use new begin_resize_drag api

We have an event, so the correct thing to do is to pass
the device into the function that we are calling. GDK
just grew a variant that takes a device, for this purpose.

https://bugzilla.gnome.org/show_bug.cgi?id=663444

Wayland: Adapt to GdkWindowImpl api change

Broadway: Adapt to GdkWindowImpl API change

Quartz: Adapt to GdkWindowImpl api change

win32: Adapt to GdkWindowImpl api change

Add device-taking variants of begin_resize/move_drag

This was one of the last places where display->core_pointer was
used in non-deprecated code paths.

GdkDeviceManager: some reshuffling of docs

tests: Disable accel test until fixed

gtk: Add test program for keycode parsing

https://bugzilla.gnome.org/show_bug.cgi?id=662755

gtk: Add accel with keycode parsing functions

Which handle accelerators with keycodes as well as keyvals,
so we can use it in applications that use GtkCellRendererAccel's
"Other" mode of operations (namely gnome-control-center and
gnome-settings-daemon).

https://bugzilla.gnome.org/show_bug.cgi?id=662755

Fix make check

introspection: Include deprecated headers for scanner too

https://bugzilla.gnome.org/show_bug.cgi?id=663345

Updated POTFILES.in

Make sure we only show icons in the file selector bookmarks toolbar

This looked really bad in non-gnome settings where default is to
show text too.

Revert a change to the plug/socket headers

I was trying to avoid including gtk.h here, but this breaks
some of our tests, and there's a good chance that 3rd party
code also relies on this include.

Remove some more uses of gdk_window_get_pointer

Remove one instance of gdk_window_get_pointer

Remove various internal uses of deprecated api

Don't warn about deprecations in deprecated/

Move deprecated gtkwidget style api to gtkstyle

Don't warn about deprecations in deprecated/

We know we are deprecated here, no need to warn.

Move gtkstyle and gtkrc to deprecated/

This required a somewhat more substantial include reshuffling.
Some typedefs have been moved to gtkiconfactory.h and to
gtksettings.h.

Move wholly deprecated files to a subdirectory

Also install their headers to a subdirectory.

Move gtk_draw_insertion_cursor out of gtkstyle.[hc]

This function does not use GtkStyle at all and is not deprecated.

Drop DISABLE_DEPRECATED guards here, too

Don't use *DISABLE_DEPRECATED guards

Instead define GDK_DISABLE_DEPRECATION_WARNINGS where appropriate.

Don't use *_SINGLE_INCLUDES defines

GLib now outlaws single includes always, anyway.

Don't use *_DISABLE_DEPRECATED

Instead of undefining the DISABLE_DEPRECATED guards,
define the GDK_DISABLE_DEPRECATION_WARNING macro where needed.
Also replace INCLUDES by AM_CPPFLAGS to shut up automake.

Don't use deprecated GValue api

gtk-demo: Don't use deprecated code

tests: Disable fontselection deprecation warnings

This test tests deprecated code...

tooltip: Don't use deprecated API

tooltip: Remove unused variables

iconview: Keep the device for dnd autoscroll

This is necessary to query the device's coordinates when doing the
scrolling.

table: Define GDK_DISABLE_DEPRECATION_WARNINGS

handlebox: Define GDK_DISABLE_DEPRECATION_WARNINGS

fontsel: Define GDK_DISABLE_DEPRECATION_WARNINGS

gdk: Move a bunch of deprecated code to a new file

The new file defines GDK_DISABLE_DEPRECATION_WARNINGS so it can happily
use deprecated APIs.

This commit moves those functions there that use deprecated functions
and currently cause warnings.

With this commit, GDK compiles without deprecation warnings.

display: Don't use deprecated functions

When synthesizing crossing events, actually query the position of the
device we care about instead of using the core pointer every time.

x11: Do fallback resize without deprecated functions

After consulting with Carlos, we agreed that it should be enough to grab
the core pointer instead of doing a full grab. If it turns out that's
wrong, we need to adapt the internal API for resizes to take the device
doing the resize.

x11: Don't use deprecated function

Use the replacement instead.

tests: Remove unused variables

box: Fix includes so hbox/vbox compile warning-free

And make sure most of the code is included before undeffing deprecated
code.

Don't use deprecated Pango api

Don't use g_value_set_char

Don't use deprecated Pango api

GDK: Adapt to GLib thread api changes

GtkSearchEngineSimple: adapt the GLib thread api changes

iconview: layout items immediately when setting a GtkTreeModel

As the draw handler expects the items to be laid out already, we cannot
queue a layout here to avoid a race condition with the resize that is
queued immediately after, which in turn would lead to a segfault later
in the paint_item() implementation.

https://bugzilla.gnome.org/show_bug.cgi?id=663138

radiotoolbutton: Fix the introspection annotations

Everything that deals with a group has to be fixed.

Plus, we have NULL checks without allowing NULL arguments; since NULL
arguments are perfectly fine, we can relax the argument checks.

https://bugzilla.gnome.org/show_bug.cgi?id=662870

radiotoolbutton: Fix the annotation for get_group()

Use the same annotation used by gtk_radio_button_get_group().

https://bugzilla.gnome.org/show_bug.cgi?id=662870

gtk: Fix XIGrabKeycode usage

A modifier of "0" is still a modifier and must be submitted normally.
A num_modifiers of 0 is essentially a noop in the server.

As mentioned by Peter Hutterer <peter.hutterer@who-t.net>

https://bugzilla.gnome.org/show_bug.cgi?id=662839

Print dialog: small keynav improvement

Mark the page range entry and the copies spin button as
activates-default.

https://bugzilla.gnome.org/show_bug.cgi?id=662670

tests: Store keycode in tree for testaccel

Useful for testing keys without a keysym.

gdk: Translate keyboard brightness keys

https://bugzilla.gnome.org/show_bug.cgi?id=662628

gdk: Make gen-keyname-table.pl work again

keyname-table.h was modified by hand, and gen-keyname-table.pl
couldn't be used to generate that file again.

https://bugzilla.gnome.org/show_bug.cgi?id=662628

Update grid tests to match

The previous fix changed some test results here.

Fix gtk_grid_attach_next_to

The function did not behave as intended when @sibling is NULL.
Also add an example to the docs to clarify the intended behaviour.

gtkrc.key.mac: add Command-cursor text navigation

and some emacs-ish Control bindings that work in native widgets.
Patch from Michael Hutchinson.
(cherry picked from commit ccf12f7b406ecbd8f0c26b0e6dc86d4593144dab)

gtkrc.key.mac: remove half the file, it was an accidential double paste
(cherry picked from commit 79a92f99a9dbdc7b1a651b0e8665807bd89c6632)

Post-release version bump

Halfway to 666 !

Bump version to 3.3.2

Updates

Fix more test breakage

a11y: Hack around infinite loops in parent setting

This is kind of a hack to get rid of infinite loops that occur when
child accessibles try to set their parent upon creation but the parent
accessible creates its children in the initialize vfunc. Because in that
case, the parent will not have an accessible set when the child tries to
access it, because it is still initializing itself. Which will cause a
new accessible to be created.

https://bugzilla.gnome.org/show_bug.cgi?id=660687

Fix make check

Somebody committed a behaviour change that broke his own
tests...tsk, tsk.

Add a css parser test for transparent

printunixdialog: fix a GtkTable->GtkGrid typo

Code was treating the GtkScrolledWindow as a grid, not the grid itself.

Fix handling of GtkCellRendererProgress::inverted

We were setting the wrong field.
https://bugzilla.redhat.com/show_bug.cgi?id=747761

notebook: update tab area allocation when switching page

When the active page is switched, it's important
gtk_notebook_pages_allocate() is called after the bulk of
gtk_notebook_real_switch_page() is run, as the former allocates a
different space according to the currently active tab, which is set by
the latter.
gtk_notebook_pages_allocate() already calls gtk_notebook_redraw_tabs()
when the allocation changes, so just move its call down to
gtk_notebook_real_switch_page() to fix the bug.

https://bugzilla.gnome.org/show_bug.cgi?id=662629

win32: fix unterminated #ifdef error

By removing all ifdefs as gtk/gtkwin32.c is only built
on win32 anyway.

https://bugzilla.gnome.org/show_bug.cgi?id=662481

tooltip: only update shape if the window is realized

If we get a composited-changed signal before the tooltip is ever realized we
have nothing to do.

This actually prevents a segfault in maybe_update_shape() because calling
gtk_widget_get_window() on a non-realized widget returns NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=662467

gtk: move _gtk_modules_has_mixed_deps() to gtkmodlesprivate.h

and remove gtkmainprivate.h completely.

gtk: clean up the private horror

- add gtkmodulesprivate.h and move stuff there from gtkprivate.h
- add gtkprivate.c and move stuff there from gtkmain.c
- add gtkwin32.c and move stuff there from gtkmain.c
- don't redefine GTK_DATADIR and friends in gtkprivate.h
- have _gtk_get_datadir() and friends on all platforms
- remove the horrid hacks where gtkprivate.h can't be included,
  or must be included later due to redefinition of the compile-time
  directories

gtkquartz: don't free the string returned by get_bundle_path()

It's statically allocated.
(cherry picked from commit c040b03c2e32a773a4d9cf4019050c2f8a5b91ce)

quartz: don't return FALSE from GdkDevice::query_state()

Not finding a child window to return is not a failure.

Updated German translation

GtkRange: use the right widget for coordinate translation

GtkRange needs to check if its allocation intersects with the resize
grip allocation (trimming its own allocation if it does).
In order to do that, it needs to translate its allocation into window
coordinates, and before that, find the window to whose the allocation
is relative; code goes all the way finding the right parent widget, but
then doesn't actually use it when translating the coordinates, leading
to using the wrong rectangles for the intersection check.

https://bugzilla.gnome.org/show_bug.cgi?id=662308

Fix document generation in out of tree builds

https://bugzilla.gnome.org/show_bug.cgi?id=661032